feat(naming): query service subscriber list #189
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
|
抱歉，当我写完代码时，发现原来已经有一个 PR（#134）了。 |
|
好的，我后面抽空看看。 如果你这个比较完善的话，就合并你这个。 |
|
看了一遍没大问题，有一个待处理项处理后就可以合并。 前一个PR是配置中心的，你这个是注册中心的其实两者不冲突。 |
|
好的，后面我会在前端项目提交一个 pr，用来显示订阅者列表 |
|
本地使用 naming-register 测试发现，一个客户端注册多个服务实例，只显示一条定阅信息。 
|
|
貌似没什么问题，naming-register 中只有一个订阅。 |
有道理，是我搞错了😂 |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Refs #73